Skip to content

Add asset embedding and property-based conformance tests#91

Merged
gvonness-apolitical merged 1 commit intomainfrom
conformance/remaining-todos
Feb 16, 2026
Merged

Add asset embedding and property-based conformance tests#91
gvonness-apolitical merged 1 commit intomainfrom
conformance/remaining-todos

Conversation

@gvonness-apolitical
Copy link
Collaborator

Summary

  • Add 4 asset embedding tests: hash verification, missing file detection, hash mismatch error, asset references in document ID
  • Add 2 property-based tests: metadata inclusion affects hash, block structure round-trip constraints
  • Update conformance matrix: 78/79 PASS, 0 TODO remaining (was 69/79 with 9 TODO)

Test plan

  • cargo test --workspace --all-features — 1,020 tests pass
  • cargo clippy --workspace --all-features -- -D warnings — 0 warnings
  • Verify conformance matrix shows 0 TODO entries

…d coverage

Closes the conformance matrix gap from 69/79 to 78/79 PASS (0 TODO).
Adds 4 asset embedding tests (hash verification, missing file detection,
hash mismatch error, asset references in document ID) and 2 property-based
tests (metadata inclusion affects hash, block structure round-trip).
@codecov
Copy link

codecov bot commented Feb 16, 2026

Codecov Report

❌ Patch coverage is 93.57143% with 9 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
cdx-core/tests/integration.rs 93.57% 9 Missing ⚠️

📢 Thoughts on this report? Let us know!

@gvonness-apolitical gvonness-apolitical merged commit c0e8442 into main Feb 16, 2026
11 checks passed
@gvonness-apolitical gvonness-apolitical deleted the conformance/remaining-todos branch February 16, 2026 16:33
@gvonness-apolitical gvonness-apolitical mentioned this pull request Feb 16, 2026
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant